Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add more camera function #318

Merged
merged 1 commit into from
Jun 17, 2023
Merged

Conversation

tbago
Copy link
Contributor

@tbago tbago commented Jun 16, 2023

1、Add start stop video subscribe in camera server.

Please check this function and don't need merge for now.

Copy link
Collaborator

@julianoes julianoes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @tbago, I think that's good. I'll also let @JonasVautherin have a review.

@julianoes julianoes merged commit 8966cf5 into mavlink:main Jun 17, 2023
@tbago
Copy link
Contributor Author

tbago commented Jun 17, 2023

ok, I have alreay implement more camera function. But I want to make the commit more clear. So I will commit multi times. So may not merge it for now.

@tbago tbago deleted the add_more_camera_function branch June 17, 2023 06:53
julianoes added a commit that referenced this pull request Oct 13, 2023
This reverts:
"camera server : add start stop video subscribe (#318)"

We will add it back in shortly. This is just because the PR following
this has not been merged.

This reverts commit 8966cf5.
julianoes added a commit that referenced this pull request Oct 13, 2023
This reverts:
"camera server : add start stop video subscribe (#318)"

We will add it back in shortly. This is just because the PR following
this has not been merged.

This reverts commit 8966cf5.
julianoes pushed a commit to tbago/MAVSDK-Proto that referenced this pull request Dec 24, 2023
julianoes added a commit that referenced this pull request Dec 24, 2023
* camera server : add start stop video subscribe (#318)

* camera_server: typos

Signed-off-by: Julian Oes <julian@oes.ch>

* camera server: add start stop video streaming, add stream id in camera side

* camera server : add subscribe camera mode

* camera server: add storage information and capture stauts subscribe

* add unit on image interval and available capacity

* storage information : add unit on read speed and write speed

* camera : add format storage and reset settings function

  * add storage id in camera side, and then the camera can support multi
    storage format.

* camera server: add respond for all camera server command

* camera_server: add video stream information

Signed-off-by: Julian Oes <julian@oes.ch>

* param_server: add subscription for changed params

Signed-off-by: Julian Oes <julian@oes.ch>

---------

Signed-off-by: Julian Oes <julian@oes.ch>
Co-authored-by: Julian Oes <julian@oes.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants